Skip to content

Conversation

@fdcavalcanti
Copy link
Contributor

Description

This change adds support for virtual e-fuses when using MCUboot.

It fixes the primary slot address in flash for the MCUBoot and sets the virtual e-fuse offset.
A new test was also added, along updates to current binaries.

Related

Testing

Tested using pytest and ESP32 target. All tests passing.


Checklist

Before submitting a Pull Request, please ensure the following:

  • 🚨 This PR does not introduce breaking changes.
  • All CI checks (GH Actions) pass.
  • Documentation is updated as needed.
  • Tests are updated or added as necessary.
  • Code is well-commented, especially in complex areas.
  • Git history is clean — commits are squashed to the minimum necessary.

@fdcavalcanti
Copy link
Contributor Author

@hfudev can we merge this on v1.x? I haven't had the time to work on 2.x.

@github-actions github-actions bot changed the title feat: add support for virtual efuse on NuttX feat: add support for virtual efuse on NuttX (RDT-1502) Sep 22, 2025
@hfudev
Copy link
Member

hfudev commented Sep 23, 2025

Hi @fdcavalcanti, I checked your PR and it looks good. There are barely any breaking changes between 1.x and 2.x. Could you rebase on the latest master and try again?

If you need a 1.x release, we can do that as well. Usually, backports should be merged after the master branch has been merged.

Besides, I noticed that there's some python 3.9 syntax in your PR, if you need to do the backport to 1.x release, we have to support python 3.7.

This is one of the major reason for us to bump to 2.x.

@fdcavalcanti fdcavalcanti changed the base branch from release/v1.x to main September 23, 2025 11:11
@fdcavalcanti
Copy link
Contributor Author

@hfudev thanks for the info! I was able to rebase and test, worked fine!

@hfudev
Copy link
Member

hfudev commented Sep 23, 2025

@fdcavalcanti no idea why the ci is not triggered. could you force push again after switching the target branch?

@hfudev hfudev merged commit 1b58435 into espressif:main Sep 23, 2025
3 of 5 checks passed
@hfudev
Copy link
Member

hfudev commented Sep 23, 2025

LGTM, thank you.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants